Skip to content
This repository has been archived by the owner on Dec 4, 2024. It is now read-only.

Filter events on server side, in order to decrease load on Marathon #541

Merged
merged 2 commits into from
Jan 29, 2018
Merged

Filter events on server side, in order to decrease load on Marathon #541

merged 2 commits into from
Jan 29, 2018

Conversation

ichernetsky
Copy link
Contributor

@ichernetsky ichernetsky commented Jan 25, 2018 •
edited by jeschkies
Loading

We introduced lightweight events with MARATHON_EE-1673. These do not
include the complete groups and make the events much smaller. This
should not only ease the consumption but also the workload for
Marathon. Furthermore Marathon LB should only request the events
it actually needs.

This patch should close MARATHON-8036 and MARATHON-8047.

@jeschkies
Copy link

Please checkout #539. We should also use the lightweight events.

@jeschkies jeschkies closed this Jan 29, 2018
@jeschkies jeschkies reopened this Jan 29, 2018
@jeschkies jeschkies mentioned this pull request Jan 29, 2018
Closed
@jeschkies jeschkies requested a review from timcharper January 29, 2018 10:51
@timcharper
Copy link
Contributor

Should we remove the client side filtering eventType filtering logic now ? It seems redundant.

@timcharper
Copy link
Contributor 

    def handle_event(self, event):
        # Server side filtering is set to event_type=api_post_event, event_type=health_status_changed_event and event_type=status_update_event"
        self.reset_from_tasks()

@drewkerrigan drewkerrigan merged commit 324b0a6 into master Jan 29, 2018
@justinrlee justinrlee mentioned this pull request Feb 5, 2018
Merged
justinrlee pushed a commit that referenced this pull request Feb 6, 2018
Filter events on server side, in order to decrease load on Marathon (#…
50c0565 
…541) (#544)

* Filter events on server side, in order to decrease load on Marathon
* Use lightweight Marathon events.
@justinrlee justinrlee mentioned this pull request Feb 6, 2018
Merged
justinrlee pushed a commit that referenced this pull request Feb 6, 2018
Filter events on server side, in order to decrease load on Marathon (#…
ef86b08 
…541) (#545)

* Filter events on server side, in order to decrease load on Marathon
* Use lightweight Marathon events.
This was referenced Feb 6, 2018
Merged
Merged
justinrlee pushed a commit that referenced this pull request Feb 6, 2018
Filter events on server side, in order to decrease load on Marathon (#…
9859943 
…541) (#547)

* Filter events on server side, in order to decrease load on Marathon
* Use lightweight Marathon events.
manriquecms pushed a commit to manriquecms/marathon-lb that referenced this pull request Feb 7, 2018
@manriquecms
Filter events on server side, in order to decrease load on Marathon (d…
c163a8e 
…2iq-archive#541)

* Filter events on server side, in order to decrease load on Marathon
* Use lightweight Marathon events.
justinrlee pushed a commit that referenced this pull request Feb 7, 2018
update beta with new changes. (#551)
c737e35 
* Filter events on server side, in order to decrease load on Marathon (#541)

* Filter events on server side, in order to decrease load on Marathon
* Use lightweight Marathon events.

* Fix build status link (#548)

* Check for MESOS_TCP (#509)

* Add long backend proxypass path test (#499)

* Add long backend proxypass path test

* Fix flake problems
manriquecms added a commit to Stratio/marathon-lb-sec that referenced this pull request Feb 8, 2018
@manriquecms
Update Marathon-LB version v1.11.3 (#17)
ce84b57 
* Change README.md (d2iq-archive#487)

Proposed change to README.md to warn users about use of zdd.py on a production environment.

* Add container syslogd support and fix 'Waiting for Pids' race condition (d2iq-archive#505)

* Add additional debug logs in reload
* Adding syslog support
* Make syslogd and retry reload configurable
* Update documentation
* Make infinite retries optional, run syslogd with runsvdir

* Update base container to resolve CVE-2017-12424 (d2iq-archive#506)

* Update base container to resolve CVE-2017-12424

Updated base container to debian:buster to resolve this attack vector.
https://security-tracker.debian.org/tracker/CVE-2017-12424

* delete unnecessary swp file

* Fix grammar in tini section of README.md (d2iq-archive#442)

* exclude bad apps from generated config (d2iq-archive#481)

exclude bad apps from generated config

* Fix reconnect logging and timing depending on error type (d2iq-archive#523)

* Fix reconnect logging and timing depending on error type

* cleanup flake8 errors

* add missing build-essential

* start at most one thread and close connection when reconnecting

* Filter events on server side, in order to decrease load on Marathon (d2iq-archive#541)

* Filter events on server side, in order to decrease load on Marathon
* Use lightweight Marathon events.

* Update official Marathon-LB version to 1.11.3

* Fix error with bash run script, percentage symbol was not escaped
drewkerrigan pushed a commit that referenced this pull request Apr 4, 2018
@drewkerrigan
update beta with new changes. (#551)
42dfa71 
* Filter events on server side, in order to decrease load on Marathon (#541)

* Filter events on server side, in order to decrease load on Marathon
* Use lightweight Marathon events.

* Fix build status link (#548)

* Check for MESOS_TCP (#509)

* Add long backend proxypass path test (#499)

* Add long backend proxypass path test

* Fix flake problems
@justinrlee justinrlee deleted the 1.11.2-filter-events branch April 27, 2018 14:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@drewkerrigan drewkerrigan drewkerrigan approved these changes

@timcharper timcharper timcharper approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ichernetsky @jeschkies @timcharper @drewkerrigan